Switch to libgsystem file reading API
authorColin Walters <walters@verbum.org>
Thu, 29 Nov 2012 21:58:39 +0000 (16:58 -0500)
committerColin Walters <walters@verbum.org>
Thu, 29 Nov 2012 21:58:39 +0000 (16:58 -0500)
More draining common utilities into libgsystem.

src/libgsystem
src/libostree/ostree-repo.c
src/libotutil/ot-gio-utils.c
src/libotutil/ot-gio-utils.h

index ab2588da8442fc6135af13ee624dd91b73a7051c..eb4cecbc1528e1d1bdd99298d6f58768c6642b24 160000 (submodule)
@@ -1 +1 @@
-Subproject commit ab2588da8442fc6135af13ee624dd91b73a7051c
+Subproject commit eb4cecbc1528e1d1bdd99298d6f58768c6642b24
index d48ebd50e3d59da50a2b0eb19c09e622408ecee1..e0c7ab68db35edf25b1d6cb2d30387b62287ab30 100644 (file)
@@ -233,7 +233,7 @@ parse_rev_file (OstreeRepo     *self,
   GError *temp_error = NULL;
   ot_lfree char *rev = NULL;
 
-  if (!ot_gfile_load_contents_utf8 (f, &rev, NULL, NULL, &temp_error))
+  if ((rev = gs_file_load_contents_utf8 (f, NULL, &temp_error)) == NULL)
     goto out;
 
   if (rev == NULL)
@@ -419,7 +419,7 @@ ostree_repo_resolve_rev (OstreeRepo     *self,
 
       if (child)
         {
-          if (!ot_gfile_load_contents_utf8 (child, &ret_rev, NULL, NULL, &temp_error))
+          if ((ret_rev = gs_file_load_contents_utf8 (child, NULL, &temp_error)) == NULL)
             {
               g_propagate_error (error, temp_error);
               g_prefix_error (error, "Couldn't open ref '%s': ", gs_file_get_path_cached (child));
index 1130225c5342a92d9c07204b82756464781bfeea..42e36dc91ea7175afa886f0b408608a208ac423b 100644 (file)
@@ -133,36 +133,6 @@ ot_gfile_get_child_build_path (GFile      *parent,
   return g_file_resolve_relative_path (parent, path);
 }
 
-gboolean
-ot_gfile_load_contents_utf8 (GFile         *file,
-                             char         **out_contents,
-                             char         **out_etag,
-                             GCancellable  *cancellable,
-                             GError       **error)
-{
-  gboolean ret = FALSE;
-  gsize len;
-  ot_lfree char *ret_contents = NULL;
-  ot_lfree char *ret_etag = NULL;
-
-  if (!g_file_load_contents (file, cancellable, &ret_contents, &len, &ret_etag, error))
-    goto out;
-  if (!g_utf8_validate (ret_contents, len, NULL))
-    {
-      g_set_error (error,
-                   G_IO_ERROR,
-                   G_IO_ERROR_INVALID_DATA,
-                   "Invalid UTF-8");
-      goto out;
-    }
-
-  ret = TRUE;
-  ot_transfer_out_value (out_contents, &ret_contents);
-  ot_transfer_out_value (out_etag, &ret_etag);
- out:
-  return ret;
-}
-
 static gboolean
 cp_internal (GFile         *src,
              GFile         *dest,
index 2481698480e636d2c27ee214458f2163e3686e17..ce055c6d5b445a6a40085a22e81c92ca3100f809 100644 (file)
@@ -42,13 +42,6 @@ GFile *ot_gfile_get_child_strconcat (GFile *parent, const char *first, ...) G_GN
 
 GFile *ot_gfile_get_child_build_path (GFile *parent, const char *first, ...) G_GNUC_NULL_TERMINATED;
 
-gboolean ot_gfile_load_contents_utf8 (GFile         *file,
-                                      char         **contents_out,
-                                      char         **etag_out,
-                                      GCancellable  *cancellable,
-                                      GError       **error);
-
-
 gboolean ot_gio_shutil_cp_a (GFile         *src,
                              GFile         *dest,
                              GCancellable  *cancellable,